Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added concepts for seasonal, conditional ratings #137

Merged
merged 12 commits into from
Sep 26, 2024

Conversation

getorymckeag
Copy link
Contributor

Contributes to (doesn't close) #14, #129.

Documents concepts, decision records.

@getorymckeag
Copy link
Contributor Author

@catkins-miso My plan here is to revisit these obviously and fill the consequences out with actual examples and spec pointers after you have that done. Unsure if it's helpful to merge this now, or wait until you're done with the work for #14

@catkins-miso
Copy link
Contributor

@getorymckeag I think your plan is to update this with examples following the approved seasonal ratings spec changes. I also think those examples would include day/night so we can close #123

getorymckeag and others added 8 commits September 26, 2024 11:41
Co-authored-by: Christopher Atkins <57670484+catkins-miso@users.noreply.github.com>
Signed-off-by: Tory McKeag <143734462+getorymckeag@users.noreply.github.com>
Co-authored-by: Christopher Atkins <57670484+catkins-miso@users.noreply.github.com>
Signed-off-by: Tory McKeag <143734462+getorymckeag@users.noreply.github.com>
Co-authored-by: Christopher Atkins <57670484+catkins-miso@users.noreply.github.com>
Signed-off-by: Tory McKeag <143734462+getorymckeag@users.noreply.github.com>
Co-authored-by: Christopher Atkins <57670484+catkins-miso@users.noreply.github.com>
Signed-off-by: Tory McKeag <143734462+getorymckeag@users.noreply.github.com>
Copy link
Contributor

@catkins-miso catkins-miso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm other than one typo

Co-authored-by: Christopher Atkins <57670484+catkins-miso@users.noreply.github.com>
Signed-off-by: Tory McKeag <143734462+getorymckeag@users.noreply.github.com>
@catkins-miso catkins-miso merged commit da622cc into trolie:1.0.0-wip Sep 26, 2024
3 checks passed
@getorymckeag getorymckeag deleted the season-concepts branch September 26, 2024 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants